Skip to content

[ISSUE #6109]🔧Update GitHub Actions deployment configuration for main branch and pull requests#6110

Merged
mxsm merged 1 commit intomainfrom
doc-6109
Feb 5, 2026
Merged

[ISSUE #6109]🔧Update GitHub Actions deployment configuration for main branch and pull requests#6110
mxsm merged 1 commit intomainfrom
doc-6109

Conversation

@mxsm
Copy link
Owner

@mxsm mxsm commented Feb 5, 2026 •
edited by coderabbitai bot
Loading

Which Issue(s) This PR Fixes(Closes)

Brief Description

How Did You Test This Change?

Summary by CodeRabbit

  • Chores
    • Modified deployment workflow to refine execution conditions and branch/path filtering.

@rocketmq-rust-robot rocketmq-rust-robot added the documentation📝 Improvements or additions to documentation label Feb 5, 2026
@rocketmq-rust-bot
Copy link
Collaborator

rocketmq-rust-bot commented Feb 5, 2026

🔊@mxsm 🚀Thanks for your contribution🎉！

💡CodeRabbit(AI) will review your code first🔥！

Note

🚨The code review suggestions from CodeRabbit are to be used as a reference only, and the PR submitter can decide whether to make changes based on their own judgment. Ultimately, the project management personnel will conduct the final code review💥.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 5, 2026 •
edited
Loading

Walkthrough

The GitHub Actions deployment workflow is reconfigured to refine trigger conditions: push events now respond to path-specific changes only, pull requests are scoped to the main branch, and deployment steps execute exclusively on non-pull-request events.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
.github/workflows/deploy.yml		 Removed master branch from push triggers; added path filters for rocketmq-website/** and .github/workflows/deploy.yml; added branches filter for main on pull requests; added conditional check to prevent deployment on pull request events.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A workflow hops to main with care,
Paths filtered here and branches there,
Push deploys, but pull requests stay,
CI/CD flows the rabbit's way! 🚀

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly references issue #6109 and clearly summarizes the main change: updating GitHub Actions deployment configuration for main branch and pull requests, which aligns with the changeset.
Linked Issues check ✅ Passed The PR implements deployment configuration updates as requested in issue #6109, adding path filters, branch conditions, and conditional deployment logic to address main branch and pull request workflows.
Out of Scope Changes check ✅ Passed All changes in the .github/workflows/deploy.yml file are directly related to updating GitHub Actions deployment configuration for main branch and pull requests as specified in issue #6109.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch doc-6109

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mxsm mxsm merged commit 2e47fc6 into main Feb 5, 2026
17 of 18 checks passed
@rocketmq-rust-bot rocketmq-rust-bot added approved PR has approved and removed ready to review waiting-review waiting review this PR labels Feb 5, 2026
@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 40.65%. Comparing base (3b8d8b6) to head (2aff65f).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #6110      +/-   ##
==========================================
- Coverage   40.65%   40.65%   -0.01%     
==========================================
  Files         887      887              
  Lines      123251   123251              
==========================================
- Hits        50111    50109       -2     
- Misses      73140    73142       +2

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mxsm mxsm deleted the doc-6109 branch February 22, 2026 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rocketmq-rust-bot rocketmq-rust-bot rocketmq-rust-bot approved these changes

@SpaceXCN SpaceXCN Awaiting requested review from SpaceXCN

@TeslaRustor TeslaRustor Awaiting requested review from TeslaRustor

Assignees

No one assigned

Labels

AI review first Ai review pr first approved PR has approved auto merge documentation📝 Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Doc📝] Update GitHub Actions deployment configuration for main branch and pull requests

3 participants

@mxsm @rocketmq-rust-bot @rocketmq-rust-robot